Skip to content

Fix compilation error on linux #88

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 17, 2019
Merged

Fix compilation error on linux #88

merged 1 commit into from
Oct 17, 2019

Conversation

spenserblack
Copy link
Collaborator

I was getting a compilation error when attempting to compile on Linux.
I tested this on both Windows and Linux, compiles on both. Will probably fail on MacOS, though.

Thanks for adding me as a collaborator, @o2sh!

@o2sh
Copy link
Owner

o2sh commented Oct 16, 2019

You can check this

This will only apply to windows

@spenserblack
Copy link
Collaborator Author

spenserblack commented Oct 17, 2019

Thanks! I used a combination of the example you provided and ansi_term's documentation to add this.
Compiles on both Linux and Windows for me.

@spenserblack spenserblack marked this pull request as ready for review October 17, 2019 13:47
@spenserblack
Copy link
Collaborator Author

Do you mind if I update .travis.yml to compile on both Linux and Windows (and possibly OSX)?

@o2sh
Copy link
Owner

o2sh commented Oct 17, 2019

Do you mind if I update .travis.yml to compile on both Linux and Windows (and possibly OSX)?

Sure 👍

@spenserblack spenserblack changed the title [WIP] Fix compilation error on linux Fix compilation error on linux Oct 17, 2019
@spenserblack
Copy link
Collaborator Author

spenserblack commented Oct 17, 2019

Travis CI is currently pointing to a cancelled build due to a force push. But this build should effectively be the same.

Looks like all builds succeeded. @o2sh, do these changes look good? If so, I'll merge this.

@o2sh
Copy link
Owner

o2sh commented Oct 17, 2019

Sure go ahead, it should fix #90

@spenserblack spenserblack merged commit f23aa63 into o2sh:master Oct 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants